Skip to content

🐛 Fix init command fails to get release#182

Closed
roziscoding wants to merge 1 commit intofrinyvonnick:masterfrom
roziscoding:bugfix/init-release
Closed

🐛 Fix init command fails to get release#182
roziscoding wants to merge 1 commit intofrinyvonnick:masterfrom
roziscoding:bugfix/init-release

Conversation

@roziscoding
Copy link
Copy Markdown

The getChangelog function used to compare options.release with from-package, but, on the init command, that option is undefined.

This changes the behaviour so that, if there's no options.release, the package version is used instead.

The `getChangelog` function used to compare `options.release` with `from-package`, but, on the `init` command, that option is undefined.

This changes the behaviour so that, if there's no `options.release`, the package version is used instead.
@roziscoding roziscoding marked this pull request as ready for review March 13, 2020 23:01
@frinyvonnick
Copy link
Copy Markdown
Owner

frinyvonnick commented Mar 25, 2020

Hi @rjmunhoz, thank you for submitting this PR 👍

This could be related to #181 ? It would be nice to have an e2e tests that ensures this error won't come again. Could you implement it? Or at least give a reproduction scenario so I could do it?

@roziscoding
Copy link
Copy Markdown
Author

Hm... I don't think it is, no.
Because this is related to when a version is not specified. I don't think this fix applies to #181, no.

About the tests, I can try to implement them, but the reproduction scenario, for me, was just trying to use the init command to create a new changelog, but I can try some different scenarios to make sure there was nothing specifically related to this in my environment.

@frinyvonnick frinyvonnick mentioned this pull request Jul 30, 2020
@frinyvonnick
Copy link
Copy Markdown
Owner

I close this pull request in favor of #204. Sorry @roziscoding I prefer a different approach to fix this issue but thank you for the proposal 🙏 It actually helps me a lot.

@roziscoding
Copy link
Copy Markdown
Author

No problem at all, @frinyvonnick !
I'm glad I could help somehow!

@frinyvonnick
Copy link
Copy Markdown
Owner

I just published it in v2.2.0 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants